Skip to content

feat: implement service link task#120

Merged
josegonzalez merged 2 commits intomasterfrom
14-implement-service-link-task
Apr 25, 2026
Merged

feat: implement service link task#120
josegonzalez merged 2 commits intomasterfrom
14-implement-service-link-task

Conversation

@josegonzalez
Copy link
Copy Markdown
Member

Summary

  • Adds dokku_service_link task that links/unlinks dokku services to/from apps, matching the upstream ansible-dokku dokku_service_link.py implementation
  • Implements idempotency via dokku <service>:linked checks and validates both service and app existence before link operations
  • Configures CI Docker daemon with DOCKER_KEEP_DEPRECATED_LEGACY_LINKS_ENV_VARS=1 for Docker 29+ legacy container link support
  • Adds runtime Docker link detection that skips integration tests when Docker doesn't support legacy links

Closes #14

Adds dokku_service_link task that links/unlinks dokku services to apps,
compatible with the upstream ansible-dokku implementation. Includes
idempotency via dokku service:linked checks and validation of both
service and app existence before attempting link operations.

Configures CI Docker daemon with DOCKER_KEEP_DEPRECATED_LEGACY_LINKS_ENV_VARS=1
to support legacy container links in Docker 29+, and adds a runtime skip
helper for environments where Docker link support is unavailable.
Verifies the service container is running via docker inspect before
linking, checks that REDIS_URL config var is set after link and cleared
after unlink, and inspects the service container hostname.
@josegonzalez josegonzalez merged commit b00ffb6 into master Apr 25, 2026
10 checks passed
@josegonzalez josegonzalez deleted the 14-implement-service-link-task branch April 25, 2026 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement service_link task

1 participant